Skip to content

JIRA cloud issue creation and availability issue#448

Merged
ralphbean merged 1 commit into
release-engineering:mainfrom
Bala-Sakabattula:fixes
Apr 2, 2026
Merged

JIRA cloud issue creation and availability issue#448
ralphbean merged 1 commit into
release-engineering:mainfrom
Bala-Sakabattula:fixes

Conversation

@Bala-Sakabattula

Copy link
Copy Markdown
Collaborator

Created this PR for solving the following issue:

  1. After jira cloud issue creation instantly if you search for through JQL it won't return so for that we have used GET call as an alternative if the key is present in cache or in snowflake but not in JQL ...

  2. As previously we are logging issue Id but it isn't that useful so we are now chnaged to the log the JQL contents it will help to debug in better way.

  3. Updated to use displayName instead of name in _update_Assignee and check_comments_for_duplicate

@webbnh webbnh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm OK with this as is, since we very much want to stop creating duplicates, but I have a couple of concerns (and a bunch of suggestions).

First, I recommend changing the focus of _get_existing_jira_issue_query() (and its name) so that it returns the key list and not the JQL -- the caller can produce the JQL easily enough, and returning a naked tuple is something best to avoid unless there's a compelling need.

Second, you modified two access to use displayName, but their respective approaches are inconsistent...which is fine if it's justified, but my instinct is that one of them is "right" and one is "wrong" -- and I don't know which is which....

So, perhaps we could merge this and then you could polish it in a follow-up PR.

Comment thread sync2jira/downstream_issue.py
Comment thread sync2jira/downstream_issue.py
Comment thread sync2jira/downstream_issue.py
Comment thread sync2jira/downstream_issue.py
Comment thread sync2jira/downstream_issue.py
Comment thread tests/test_downstream_issue.py
Comment thread tests/test_downstream_issue.py
@ralphbean ralphbean enabled auto-merge (squash) April 2, 2026 18:10
@ralphbean ralphbean merged commit bb89d28 into release-engineering:main Apr 2, 2026
6 checks passed
Bala-Sakabattula added a commit that referenced this pull request Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants